Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Do not support old versions of Rails + Ruby / Prepare for 4.0.0 #129

Merged
merged 5 commits into from
Jun 24, 2024
Merged

Do not support old versions of Rails + Ruby / Prepare for 4.0.0 #129

merged 5 commits into from
Jun 24, 2024

Conversation

ssunday
Copy link
Contributor

@ssunday ssunday commented Jun 24, 2024

#128 (comment)

By submitting this pull request, I confirm that my contribution is made under
the terms of the Apache 2.0 license.

@ssunday
Copy link
Contributor Author

ssunday commented Jun 24, 2024

Do not see anything to update in README on first glance

@mullermp
Copy link
Contributor

In README, change to this

gem 'aws-sdk-rails', '~> 4'

Since we are MV bumping this semi regularly now with new Rails releases, I think it's reasonable to have people pin to MV.

sample_app/Gemfile.lock Outdated Show resolved Hide resolved
VERSION Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
.github/workflows/ci.yml Outdated Show resolved Hide resolved
@mullermp
Copy link
Contributor

I pushed an update for this.

@ssunday
Copy link
Contributor Author

ssunday commented Jun 24, 2024

@mullermp beat me to the punch!

@mullermp
Copy link
Contributor

Some JRuby test is hanging. We can fix this in the Rspec conversion.

@mullermp mullermp merged commit fcd058b into aws:main Jun 24, 2024
15 of 18 checks passed
@ssunday ssunday deleted the drop-old-versions branch June 24, 2024 15:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants